Skip to content

Remove old way of HALT Reboot method#1

Merged
ndas7 merged 1 commit intondas7:gnoi_rebootfrom
vvolam:gnoi_reboot_halt
Feb 14, 2025
Merged

Remove old way of HALT Reboot method#1
ndas7 merged 1 commit intondas7:gnoi_rebootfrom
vvolam:gnoi_reboot_halt

Conversation

@vvolam
Copy link

@vvolam vvolam commented Jan 25, 2025

Why I did it

How I did it

How to verify it

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111

Description for the changelog

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

Copy link

@hdwhdw hdwhdw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM but can we leave some trace in the PR description for why we are doing this? Maybe also link to the PR that add the new method?

@vvolam
Copy link
Author

vvolam commented Jan 31, 2025

LGTM but can we leave some trace in the PR description for why we are doing this? Maybe also link to the PR that add the new method?

Since this will be stacked with the base ndas7:gnoi_reboot PR, it goes along with the new PR where the new method is added.

@ndas7
Copy link
Owner

ndas7 commented Feb 14, 2025

Please resolve conflicts.

@vvolam
Copy link
Author

vvolam commented Feb 14, 2025

Please resolve conflicts.

@ndas7 Its fixed. Thank you!

@ndas7 ndas7 merged commit c3a58eb into ndas7:gnoi_reboot Feb 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants